Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configure e2e tests for OTel collector #5270

Merged
merged 28 commits into from
Aug 22, 2024
Merged

Conversation

KevinFairise2
Copy link
Member

@KevinFairise2 KevinFairise2 commented Aug 1, 2024

Description:

This PR configures the repository to run e2e tests. It creates all the setup code to install the OTel collector built in the pipeline using helm chart and adds a job to publish the image on the agent-qa registry.

This PR also introduces basic e2e tests for OTLP traces, metrics, and logs.

Link to tracking Issue:
https://datadoghq.atlassian.net/browse/OTEL-1994

Testing:

Documentation:
https://datadoghq.atlassian.net/wiki/spaces/OTEL/pages/4013621253/OpenTelemetry+Collector+E2E+Tests

@KevinFairise2 KevinFairise2 changed the title Add a job that publish otel-collector on agent qa registry Configure e2e tests for OTel collector Aug 1, 2024
@KevinFairise2 KevinFairise2 marked this pull request as ready for review August 1, 2024 11:24
.gitlab-ci.yml Show resolved Hide resolved
.gitlab-ci.yml Outdated Show resolved Hide resolved
e2e-tests/otel-collector/provisioner.go Show resolved Hide resolved
e2e-tests/tests/otel_test.go Show resolved Hide resolved
e2e-tests/otel-collector/helm/values.yaml Outdated Show resolved Hide resolved
@liustanley
Copy link

If possible, could you also port over the inv new-e2e-tests.clean -s script into this repository in case you're running tests locally and need to clean local state?

@KevinFairise2
Copy link
Member Author

If possible, could you also port over the inv new-e2e-tests.clean -s script into this repository in case you're running tests locally and need to clean local state?

If possible I'd like to avoid doing that because we'd need to bring a lot of python dependencies on that repo that does not use Python.
However if you run inv new-e2e-tests.clean -s from datadog-agent it should clean all your Pulumi stacks, including the one created from the test run on that repository!

@liustanley liustanley requested review from liustanley and removed request for liustanley August 14, 2024 18:32
@liustanley liustanley requested a review from dineshg13 August 14, 2024 18:36
e2e-tests/otel-collector/helm/otel_collector.go Outdated Show resolved Hide resolved
.github/CODEOWNERS Show resolved Hide resolved
@liustanley liustanley merged commit 03b63e6 into prod Aug 22, 2024
148 of 155 checks passed
@liustanley liustanley deleted the kfairise/new-e2e-test-collector branch August 22, 2024 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants